Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Instruct to install imagemagick directly, without development packages #643

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vsalvino
Copy link

@vsalvino vsalvino commented Dec 8, 2023

See issue #642

Confirmed pip install wand and >>> from wand.image import Image works correctly based on the update installation methods on Debian and Fedora.

Since the user of the wand python package is not expected to be actively compiling an imagemagick C library, the development dependencies are not necessary. Only the actual imagemagick end-user library.

@emcconville
Copy link
Owner

Thanks!

Please drop those Docker paragraphs below the Debian & Fedora installs.

If you're using in a Docker image, it may be beneficial to avoid installing
extra recommended packages:

The Docker image, beneficial and non-essential are opinions of a common virtualization use-case, and is not the roll of this Installation guide to comment on such. Best just keep everything generic. Dropping the devel dependencies are enough.

@vsalvino
Copy link
Author

vsalvino commented Dec 8, 2023

Happy to remove it, just wanted to share my reasoning below first.

As someone who works with juniors or newcomers frequently, in my experience most people using Docker do not know how to optimize for Docker, and have little understanding of Linux. Given that a huge percentage of Python is run in Docker images, I figured it would be beneficial to share this tidbit rather than assume the reader already knows the best installation method. The problem is compounded these days by Artificial Unintelligence that will encourage people to copy-paste stuff straight from the search results without thinking about it. By adding this content we can help steer the search engines in the right direction as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants